Skip to content

Conversation

@caseyvaughn
Copy link
Collaborator

@caseyvaughn caseyvaughn commented Sep 13, 2025

Add basic contact information to Facilitator to match requirements from Collaboration Agreement.
Issue #145

  • add new columns to Facilitator for contact information fields
  • drop email column (replaced with primary_email_address)
  • form styling will be tackled later after Rails updates have gone through & css/sass styling is stable

Facilitator show page with new contact information:
Screenshot 2025-09-13 at 2 07 30 PM

@caseyvaughn caseyvaughn marked this pull request as ready for review September 13, 2025 18:31
it { should validate_presence_of(:email) }
it { should validate_presence_of(:primary_email_address) }
it { should validate_presence_of(:primary_email_address_type) }
it { should validate_presence_of(:street_address) }
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we are requiring all of these fields?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The collaboration agreement has * by all these fields so I'm assuming they are all required. I can confirm with the AWBW team this afternoon! If they're not required I'll change these

Copy link
Collaborator

@maebeale maebeale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🏅 thanks for moving this along, Casey!

@caseyvaughn caseyvaughn force-pushed the add-contact-info-to-facilitators branch from bca24f8 to 9fabd80 Compare September 13, 2025 19:49
@caseyvaughn caseyvaughn merged commit d6ceb03 into main Sep 13, 2025
2 checks passed
@caseyvaughn caseyvaughn deleted the add-contact-info-to-facilitators branch September 13, 2025 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants